Skip to content

feat: Git Configure Edit modal #806

Merged
dennisvankekem merged 8 commits into
mainfrom
APL-1945
Jun 22, 2026
Merged

feat: Git Configure Edit modal #806
dennisvankekem merged 8 commits into
mainfrom
APL-1945

Conversation

@dennisvankekem

@dennisvankekem dennisvankekem commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

console: #806
api: linode/apl-api#1017

Considerations

  • I have tested the changes in both light and dark mode.
  • I have considered the need for new unit tests.
  • I have tested the changes on a cluster.
  • I have included relevant documentation updates.
  • I have an approved Figma design or have reflected my changes in Figma
  • I have verified that the UI/UX is consistent in major browsers (e.g., Chrome, Firefox, Safari, Edge).
  • I have tested the changes for responsiveness in different screen resolutions.
  • I have tested expected error states and verified that the user is presented with informative error messages.
  • I have tested the feature with unusual or extreme inputs (e.g., very long strings, empty states, clicking a button multiple times quickly).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an “Edit/Configure Git” experience in the console by fetching current Git settings, conditionally pre-populating the modal, and exposing copy-to-clipboard affordances.

Changes:

  • Add a new RTK Query endpoint/hook to GET current Git settings (/v2/git) and extend Git-related types.
  • Update the Git settings modal to load existing configuration, show an internal-repo URL block, and add copy buttons.
  • Rename the Settings tile from “Git” to “GitOps” and hide the otomi.git section in the generic settings form UI schema.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
src/redux/otomiApi.ts Adds getGitSettings endpoint/hook and expands Git settings typings.
src/pages/SettingsOverview.tsx Renames the settings card label from “Git” to “GitOps”.
src/components/Setting.tsx Hides otomi.git in the settings form UI schema.
src/components/modals/ConfigureGitModal.tsx Fetches current Git settings, pre-fills the form, adds copy-to-clipboard UI, and adjusts modal flow.
src/components/InformationBanner.tsx Tweaks spacing between the info icon and message text.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/components/modals/ConfigureGitModal.tsx
Comment thread src/redux/otomiApi.ts
Comment thread src/redux/otomiApi.ts
Comment thread src/components/modals/ConfigureGitModal.tsx
Comment thread src/components/modals/ConfigureGitModal.tsx
Comment thread src/components/modals/ConfigureGitModal.tsx Outdated
Comment thread src/components/modals/ConfigureGitModal.tsx
svcAPLBot and others added 5 commits June 18, 2026 14:10
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread src/components/modals/ConfigureGitModal.tsx
@dennisvankekem dennisvankekem merged commit 2a50c26 into main Jun 22, 2026
6 checks passed
@dennisvankekem dennisvankekem deleted the APL-1945 branch June 22, 2026 14:17
j-zimnowoda pushed a commit that referenced this pull request Jun 25, 2026
Co-authored-by: svcAPLBot <174728082+svcAPLBot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants